Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: Add conventional-precommit-linter (hook) #814

Merged
merged 1 commit into from
Sep 5, 2023
Merged

ci: Add conventional-precommit-linter (hook) #814

merged 1 commit into from
Sep 5, 2023

Conversation

kolipakakondal
Copy link
Collaborator

@kolipakakondal kolipakakondal commented Sep 5, 2023 •
edited by coderabbitai bot
Loading

Description

ci: Add conventional-precommit-linter (hook)

Fixes # (IEP-2020)

Type of change

  • New feature (non-breaking change which adds functionality)

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

  • New Feature: Added a pre-commit configuration file. This will help maintain code quality by running a conventional commit linter before each commit, ensuring that all commit messages adhere to a consistent standard. This change enhances the development process and contributes to better version control management.

@coderabbitai
Copy link

coderabbitai bot commented Sep 5, 2023 •
edited
Loading

Walkthrough

This change introduces a pre-commit configuration file that sets up a conventional commit linter. This linter will run at the commit-msg stage, ensuring all commit messages adhere to a standard format.

Changes

File Summary
.pre-commit-config.yaml Added a new configuration file for pre-commit hooks. It specifies the minimum version of pre-commit and default hook types. A new repository for a conventional commit linter is included with a specific revision.

🎉🐇

With each commit we make,

Let's ensure no rule we break.

For consistency's sake,

A linter we now partake! 📝💫

Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • Mention @coderabbitai in any review comment for bot assistance. The bot stays silent unless called upon.
  • Note: Review comments are made on code diffs or files, not on the PR overview.

Pause Incremental Reviews

  • Insert @coderabbitai: ignore in the PR description to halt the bot's ongoing reviews. Remove the line to resume.

coderabbitai[bot]
coderabbitai bot reviewed Sep 5, 2023
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review Status

Actionable comments generated: 0

Commits Files that changed from the base of the PR and between cb692fa and 1828b88 commits.
Files selected for processing (1)
  • .pre-commit-config.yaml (1 hunks)
Files skipped from review due to trivial changes (1)
  • .pre-commit-config.yaml

@kolipakakondal kolipakakondal merged commit fb53f00 into master Sep 5, 2023
@kolipakakondal kolipakakondal deleted the IEP-1020 branch September 5, 2023 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kolipakakondal